Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ticket 133: added optional hq_address_street_3 and hq_address_street_… #134

Conversation

nargis-sultani
Copy link
Contributor

…4 fields to financial_institutions table
closes #133

Copy link

github-actions bot commented Apr 9, 2024

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  src/regtech_user_fi_management/entities/models
  dao.py
  dto.py
Project Total  

This report was generated by python-coverage-comment-action

@nargis-sultani nargis-sultani self-assigned this Apr 9, 2024
Copy link
Collaborator

@lchen-2101 lchen-2101 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lchen-2101 lchen-2101 merged commit 8c7d897 into main Apr 9, 2024
4 checks passed
@lchen-2101 lchen-2101 deleted the features/133_add_hq_address_state_3_and_4_fields_to_fi_table branch April 9, 2024 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add optional hq_address_street_3 and hq_address_street_4 fields
2 participants